add banner and facts #1
Conversation
index.html
Outdated
| </div> | ||
| </nav> | ||
| </div> | ||
| <div class="row box-banner "> |
There was a problem hiding this comment.
пробел лишний, из-за него бывают проблемы
There was a problem hiding this comment.
пробел так и остался
| @@ -0,0 +1,52 @@ | |||
| .about { | |||
| &__text-top-l { | |||
There was a problem hiding this comment.
что за приставки -r и -l везде?
scss/_about.scss
Outdated
| top: -30px; | ||
| left: -30px; | ||
| } | ||
|
|
scss/_about.scss
Outdated
| @include font("Source Sans Pro", 16, rgba(255, 255, 255, 0.5)); | ||
| font-weight: 400; | ||
| } | ||
|
|
There was a problem hiding this comment.
тоже лишняя строка
| @@ -0,0 +1,42 @@ | |||
| .facts { | |||
| background: rgba( 253, 242, 255, 1.0); | |||
| ; | |||
There was a problem hiding this comment.
ну у этого блока не белый фон а такой в фигме...
There was a problem hiding this comment.
не, я про точку с запятой на третьей строке
кстати, на счет фона, если прозрачности нет, то rgba() просто не используют, а вместо него rgb
scss/_facts.scss
Outdated
| &__boxes { | ||
| flex-wrap: wrap; | ||
| justify-content: center; | ||
|
|
There was a problem hiding this comment.
3 лишних строки
There was a problem hiding this comment.
блин, привычка когда стиль пишу эмет предлагает значения а потом для перехода на следующую строку Enter жму на автомате постоянно.
There was a problem hiding this comment.
один энтер не добавляет 3 лишних строки)))
scss/_facts.scss
Outdated
| } | ||
| &__text { | ||
| @include font("Source Sans Pro", 16, rgba(51, 51, 51, 0.5)); | ||
|
|
There was a problem hiding this comment.
а ты любишь лишние строки
scss/_facts.scss
Outdated
|
|
||
| .icon-photo { | ||
| font-size: 42px; | ||
|
|
There was a problem hiding this comment.
парм-пам-пам лшиняя строка
scss/_facts.scss
Outdated
| .icon-0 { | ||
| font-size: 42px; | ||
|
|
||
| }.icon-health { |
There was a problem hiding this comment.
как это на одной строке с закрывающей скобочкой оказался новый селектор?
There was a problem hiding this comment.
селектор все так же и остался)
| @import "_variables"; | ||
| @import "_facts"; | ||
| @import "_about"; | ||
| html { |
There was a problem hiding this comment.
а вот тут лучше бы лишнюю строку)
| } | ||
| } | ||
| &__blog { | ||
| font-weight: 600; |
There was a problem hiding this comment.
а чего не через миксин задаешь толщину шрифта?
scss/styles.scss
Outdated
| } | ||
| } | ||
| .arrow-left { | ||
| width: 48px; |
There was a problem hiding this comment.
ого сколько стилей повторяющихся. DRY
No description provided.